hw: Add optional hardwired cluster alias region #52
Conversation
paulsc96
changed the title
paulsc96
force-pushed
the
paulsc/tcdm-alias
branch
4 times, most recently
from
07db011 to
e63c654
Compare
There was a problem hiding this comment.
Otherwise looks good to me :)
Should I add a test to verify the functionality? This would require enabling the alias in the default cluster configuration, as the CI can currently test only one configuration. Otherwise we can extend the CI to test multiple configurations, which I think will be required in the long term, but we should discuss implementation details.
colluca
force-pushed
the
paulsc/tcdm-alias
branch
from
edab995 to
75ff47c
Compare
|
To keep track of the status, only Banshee tests are failing as this architectural feature is yet to be modeled in Banshee. |
|
As requested, this should be made configurable instead. Will look into that next. |
After clarifying this, it shouldn't be necessary anymore. |
paulsc96
force-pushed
the
paulsc/tcdm-alias
branch
from
75ff47c to
30ac4d6
Compare
* docs: Update docker information * docs: Update README to use image uploaded to ghcr.io * ci: Update CI to use image uploaded in ghcr.io
paulsc96
force-pushed
the
paulsc/tcdm-alias
branch
from
18ec5af to
d3b90f5
Compare
|
@fischeti could you also double-check this PR? Then we can merge it. |
This adds an optional hardwired alias region to the
snitch_clusterin addition to the pseudostaticcluster_base_addr_i. This is useful in multi-cluster systems (e.g. Occamy) to enable static linking and optimization of TCDM contents.Along the way, I removed some unconnected address inputs from
snitch_cluster_peripheral.For Occamy, I recommend enabling the prefix and mapping it to
0x1800_0000.